feat: remove precommit hooks & move to ci #320
Conversation
…tions - Removed husky and lint-staged from package.json and package-lock.json. - Added new scripts for checking code formatting and documentation. - Updated GitHub Actions workflow to include formatting and documentation checks. - Cleaned up package-lock.json by removing unused dependencies.
Le-Caignec
changed the title
Le-Caignec
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR removes Husky and lint-staged git hooks in favor of CI-based checks for code formatting and documentation updates. The changes migrate pre-commit checks (code formatting via prettier and documentation generation) to GitHub Actions workflow steps, ensuring consistency and reducing local development dependencies.
Key changes:
- Removed husky and lint-staged dependencies and configuration
- Added
check-formatscript to verify prettier formatting without modifying files - Added
check-docscript to verify documentation is up-to-date - Integrated both checks into the CI workflow
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Removed husky prepare script, lint-staged config, added check-format and check-doc scripts |
| package-lock.json | Removed husky, lint-staged and their transitive dependencies |
| .husky/pre-commit | Deleted git hook file that ran lint-staged and conditional doc generation |
| .github/workflows/main.yml | Added format checking and documentation verification steps to CI pipeline |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #320 +/- ##
=======================================
Coverage 96.53% 96.53%
=======================================
Files 32 32
Lines 1069 1069
Branches 213 199 -14
=======================================
Hits 1032 1032
Misses 37 37 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Uh oh!
There was an error while loading. Please reload this page.